Skip to content

[LOCAL][0.83] Add redbox subspec to React-debug pod#56584

Merged
motiz88 merged 1 commit intofacebook:0.83-stablefrom
motiz88:fix-react-debug-header-mappings
Apr 23, 2026
Merged

[LOCAL][0.83] Add redbox subspec to React-debug pod#56584
motiz88 merged 1 commit intofacebook:0.83-stablefrom
motiz88:fix-react-debug-header-mappings

Conversation

@motiz88
Copy link
Copy Markdown
Contributor

@motiz88 motiz88 commented Apr 23, 2026

Summary

Add a redbox subspec to React-debug.podspec with its own header_dir, following the same pattern React-Fabric uses for its subdirectories.

Without this, static library builds (without USE_FRAMEWORKS) flatten all headers into the top-level header_dir (react/debug), so imports like <react/debug/redbox/RedBoxErrorParser.h> fail because the redbox/ subdirectory is lost. This broke the test_e2e_ios_templateapp job after the RedBox 2.0 backport (#56574) added files under ReactCommon/react/debug/redbox/.

Changelog: [Internal]

Test Plan

CI — test_e2e_ios_templateapp should no longer fail with 'react/debug/redbox/RedBoxErrorParser.h' file not found.

@meta-cla meta-cla Bot added the CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. label Apr 23, 2026
@github-actions
Copy link
Copy Markdown

Warning

JavaScript API change detected

This PR commits an update to ReactNativeApi.d.ts, indicating a change to React Native's public JavaScript API.

  • Please include a clear changelog message.
  • This change will be subject to additional review.

This change was flagged as: BREAKING

@motiz88 motiz88 force-pushed the fix-react-debug-header-mappings branch from 2ed6375 to d7e7efb Compare April 23, 2026 17:03
Add a "redbox" subspec to React-debug.podspec with its own header_dir,
following the same pattern React-Fabric uses for its subdirectories.

Without this, static library builds (without USE_FRAMEWORKS) flatten all
headers into the top-level header_dir ("react/debug"), so imports like
`<react/debug/redbox/RedBoxErrorParser.h>` fail because the "redbox/"
subdirectory is lost.
@motiz88 motiz88 force-pushed the fix-react-debug-header-mappings branch from d7e7efb to e639267 Compare April 23, 2026 17:04
@motiz88 motiz88 changed the title [0.83] Preserve header subdirectory structure in React-debug pod [0.83] Add redbox subspec to React-debug pod Apr 23, 2026
@motiz88 motiz88 changed the title [0.83] Add redbox subspec to React-debug pod [LOCAL][0.83] Add redbox subspec to React-debug pod Apr 23, 2026
@motiz88 motiz88 marked this pull request as ready for review April 23, 2026 17:17
@motiz88 motiz88 merged commit 5c63f8d into facebook:0.83-stable Apr 23, 2026
32 of 39 checks passed
motiz88 added a commit to motiz88/react-native that referenced this pull request Apr 28, 2026
Add a "redbox" subspec to React-debug.podspec with its own header_dir,
following the same pattern React-Fabric uses for its subdirectories.

Without this, static library builds (without USE_FRAMEWORKS) flatten all
headers into the top-level header_dir ("react/debug"), so imports like
`<react/debug/redbox/RedBoxErrorParser.h>` fail because the "redbox/"
subdirectory is lost.
meta-codesync Bot pushed a commit that referenced this pull request Apr 28, 2026
Summary:
Same fix as #56584 (targeting 0.83-stable), applied to main.

Add a `redbox` subspec to `React-debug.podspec` with its own `header_dir`, following the same pattern `React-Fabric` uses for its subdirectories.

Without this, static library builds (without `USE_FRAMEWORKS`) flatten all headers into the top-level `header_dir` (`react/debug`), so imports like `<react/debug/redbox/RedBoxErrorParser.h>` fail because the `redbox/` subdirectory is lost.

On main this is currently masked because `test_e2e_ios_templateapp` uses prebuilt `React.framework`, but it would break any from-source static library build that includes `React-CoreModules`.

Changelog: [Internal]

Pull Request resolved: #56585

Test Plan: CI

Reviewed By: cortinico

Differential Revision: D102809818

Pulled By: motiz88

fbshipit-source-id: 9871706a8e028c27f64f4d0dd9942d7c408c6b3b
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. p: Facebook Partner: Facebook Partner Pick Request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant